Skip to content

evp_rand: add freeze option#81

Closed
andrewkdinh wants to merge 1 commit into
openssl:mainfrom
andrewkdinh:evp_rand_freeze
Closed

evp_rand: add freeze option#81
andrewkdinh wants to merge 1 commit into
openssl:mainfrom
andrewkdinh:evp_rand_freeze

Conversation

@andrewkdinh

Copy link
Copy Markdown
Contributor

Adds -f option to evp_rand perftool to freeze OSSL_LIB_CTX

$ ./evp_rand -o evp_shared 10
Average time per computation: 3.014978us
$ ./evp_rand -o evp_shared -f 10
Average time per computation: 1.696625us
$ ./evp_rand -o evp_isolated 10
Average time per computation: 18.306670us
$ ./evp_rand -o evp_isolated -f 10
Average time per computation: 13.355264us

Speeds up evp_rand_fetch, even for evp_shared case

Fixes openssl/project#1881

@Sashan Sashan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me, thanks.

@jogme jogme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@andrewkdinh andrewkdinh marked this pull request as ready for review March 2, 2026 16:32
@andrewkdinh andrewkdinh requested a review from nhorman March 2, 2026 16:35
npajkovsky pushed a commit that referenced this pull request Mar 2, 2026
Reviewed-by: Saša Nedvědický <sashan@openssl.org>
Reviewed-by: Norbert Pocs <norbertp@openssl.org>
Reviewed-by: Nikola Pajkovsky <nikolap@openssl.org>
MergeDate: Mon Mar  2 18:24:26 2026
(Merged from #81)
@npajkovsky

Copy link
Copy Markdown

Merged. Thank you for your contribution.

@npajkovsky npajkovsky closed this Mar 2, 2026
Sashan pushed a commit to Sashan/perftools that referenced this pull request Mar 8, 2026
Reviewed-by: Saša Nedvědický <sashan@openssl.org>
Reviewed-by: Norbert Pocs <norbertp@openssl.org>
Reviewed-by: Nikola Pajkovsky <nikolap@openssl.org>
MergeDate: Mon Mar  2 18:24:26 2026
(Merged from openssl#81)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add frozen flag to evp_rand perftool

4 participants